Skip to content

Add Pipeline hashing, saving, loading, list, and delete #118

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 94 commits into from
Oct 21, 2025
Merged

Add Pipeline hashing, saving, loading, list, and delete #118

merged 94 commits into from
Oct 21, 2025

Conversation

@Synicix
Copy link
Contributor

@Synicix Synicix commented Oct 13, 2025 •
edited
Loading

  • Pipeline now has hash and annotation
  • Store now implements save, load, list, and delete for pipeline
  • Change all assert_eq to pretty assert
  • Add dot_lex and get_kernel_lut for pipeline hashing.

Depends on #116

Resolve PIPE-252

synicix and others added 30 commits August 4, 2025 23:38
Move to dev
844fdff
@Synicix
Remove uneeded results and change to return iterator
764387d
@Synicix
Remove accident comment
a03db37
@Synicix
Add ground work for pod operator
abbb5ce
@Synicix
Add async operation for JoinOperator
450a90f
@Synicix
Add pod operator + better error handling for pod_result:::new()
e4b3a54
@Synicix
Improve error handling for file io in localfilestore
fee8d9c
@Synicix
Update operator to be in operation again
c04335f
@Synicix
Fix pod result hashing bug
ebe04d1
@Synicix
Apply feedback from review
20f2345
@Synicix
Merge branch 'worktree-docker-patch' into 106
51dfea6
@Synicix
Remove stale error
cb72829
@Synicix
Merge branch 'worktree-docker-patch' into 106
d36ec71
@Synicix
Save changes
2986e0d
@Synicix
Merge branch '106' into pipeline_runner
0e3dc71
@Synicix
Split model in core to match uniffi
595a12e
@Synicix
Merge branch 'model_split' into 106
b671dba
@Synicix
Merge branch '106' into runner
ac363f2
@Synicix
Save progress
86d5f70
@Synicix
Merge branch 'dev' into model_split
aead411
@Synicix
Fix missing stuff
58ba198
@Synicix
Merge branch 'model_split' into runner
9c72bd8
@Synicix
save change
7ee5251
@Synicix
Update pipeline_runner to use new operator + improvments
fda43aa
@Synicix
Readded tests
1e9a19f
@Synicix
save progress
32299f8
@Synicix
Merge remote-tracking branch 'upstream' into runner
e93c32b
@Synicix
Fix logic bug
ab900b6
@Synicix
Update tests and test fixture to merge sentence correctly
4f511ce
@Synicix
Fix agent event to make it more efficient
2a4f1f7
@Synicix Synicix requested a review from eywalker October 13, 2025 04:07
@Synicix Synicix added the enhancement New feature or request label Oct 13, 2025
@codecov
Copy link

codecov bot commented Oct 13, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 88.23912% with 181 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/core/pipeline_runner.rs 81.48% 92 Missing ⚠️
src/uniffi/store/filestore.rs 80.53% 29 Missing ⚠️
src/core/error.rs 0.00% 14 Missing ⚠️
src/core/model/pipeline.rs 95.40% 14 Missing ⚠️
src/uniffi/model/pod.rs 85.41% 7 Missing ⚠️
src/core/operator.rs 96.70% 6 Missing ⚠️
src/uniffi/model/packet.rs 68.42% 6 Missing ⚠️
src/core/crypto.rs 89.28% 3 Missing ⚠️
src/core/store/filestore.rs 93.87% 3 Missing ⚠️
src/uniffi/model/pipeline.rs 96.20% 3 Missing ⚠️
... and 2 more

📢 Thoughts on this report? Let us know!

@eywalker eywalker merged commit d56b4e3 into nauticalab:dev Oct 21, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eywalker eywalker eywalker approved these changes

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Synicix @eywalker